home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part2 / 15524 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  3.0 KB
  1. Path: news.ahc.ameritech.com!datalytics!usenet
  2. From: Rob Stewart <stew@datalytics.com>
  3. Newsgroups: comp.lang.c++
  4. Subject: Re: (no subject)
  5. Date: Fri, 05 Apr 1996 18:44:07 -0500
  6. Organization: Datalytics, Inc
  7. Message-ID: <3165B047.12E6@datalytics.com>
  8. References: <4juvuk$4vn@newserv.ksu.ksu.edu>
  9. NNTP-Posting-Host: 204.62.224.71
  10. Mime-Version: 1.0
  11. Content-Type: text/plain; charset=us-ascii
  12. Content-Transfer-Encoding: 7bit
  13. X-Mailer: Mozilla 2.0 (WinNT; I)
  14.  
  15. Iyer wrote:
  16. > 
  17. > I have a problem in returning a two dimensional array.I have declared it
  18. > as a two dimensional pointer in my Matrix class.This is a private
  19. > variable.To access this i have two functions .One is
  20. > void SetMatrixElements(double **mat)
  21. > double **GetMatElements();
  22. > When i receive this GetMatElements(),i am declaring a double **temp,
  23. > then I am receiving in this temp pointer.But i  am not allocating any
  24. > memory for this temp pointer.When i delete the the Matrix object, it's
  25. > mat elements get deleted,but to which this temp is pointing.But i cannot
  26. > delete the temp before i quit the function,this will delete the mat
  27. > elements.So, what shall i do.
  28. > 
  29. > class Matrix
  30. > {
  31. >   private:
  32. > int nc;int nr; double **mat;
  33. > public:
  34. > double **GetMatElemnts();
  35. > void SetMatElemnts(double **m);
  36. > 
  37. > }
  38. > 
  39. > Matrix *A;
  40. > void Function(){
  41. > double **temp;
  42. > temp = A->GetMatElements();
  43. > 
  44. > }
  45.  
  46. Returning a non-const pointer to a private dm is poor practice.  
  47. That aside, the problem is that GetMatElements is returning a 
  48. pointer to the same heap allocation as Matrix::mat points to.  
  49. If you choose to retain the current design, you will need 
  50. GetMatElements to allocate and populate a second array as a copy 
  51. of mat.  It can then return the pointer to this copy.  As 
  52. parallel behavior, SetMatElements must first "delete [] mat" 
  53. before assigning m to mat.
  54.  
  55. This implementation causes an ownership problem.  If you call 
  56. GetMatElements to get a pointer to an array, modify one or more 
  57. elements, then call SetMatElements with that pointer, who owns 
  58. the memory?  The caller has a pointer from GetMatElements which 
  59. is known to be a copy of Matrix's mat data member.  Therefore, 
  60. the caller should "delete [] temp."  However, the caller used 
  61. that pointer (temp) in a call to SetMatElements.  SetMatElements 
  62. then deleted its previous matrix and set mat to point to the 
  63. matrix given to it.  Therefore, SetMatElements took ownership of 
  64. the memory.
  65.  
  66. The only safe implementation for this is to use a vector class 
  67. to manage the data.  vector is part of STL, so it's reasonable 
  68. to expect that you don't have it available.  However, you can 
  69. write your own one dimensional array class overloading operator 
  70. [] to access the elements.  The result would be the following 
  71. declaration:
  72.  
  73.     vector<vector<double>> matrix;
  74.  
  75. To access a specific element, you simply do this:
  76.  
  77.     matrix[n][m]
  78.  
  79. Now, you can read and write individual elements with clear 
  80. syntax just as you would write them on paper.
  81.  
  82. -- 
  83. Robert Stewart        | My opinions are usually my own.
  84. Datalytics, Inc.    | stew@datalytics.com
  85.